[google_maps_flutter] Add cameraControl enable/disable & position on web #9089

4rthurmonteiro · 2025-04-15T19:30:39Z

Add support to camera control button on web.

Pre-Review Checklist

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling. ↩ ↩² ↩³

harryterkelsen

Thanks for your contribution! This PR looks good to me with a minor nit about improving doc comments. This is ready to split into separate PRs for landing

harryterkelsen · 2025-06-11T21:14:38Z

packages/google_maps_flutter/google_maps_flutter/lib/src/google_map.dart

+  /// See [WebCameraControlPosition] for more details.
+  final WebCameraControlPosition? webCameraControlPosition;
+
+  /// This setting controls how the API handles cameraControl button on the map. Web only.


Have a more descriptive comment here. Something like "Enables or disables the Camera controls."

harryterkelsen

LGTM

packages/google_maps_flutter/google_maps_flutter/lib/src/google_map.dart

stuartmorgan-g · 2025-06-12T12:09:25Z

packages/google_maps_flutter/google_maps_flutter/pubspec.yaml

@@ -2,7 +2,7 @@ name: google_maps_flutter
 description: A Flutter plugin for integrating Google Maps in iOS and Android applications.
 repository: https://github.com/flutter/packages/tree/main/packages/google_maps_flutter/google_maps_flutter
 issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+maps%22
-version: 2.12.1
+version: 2.12.2


As indicated in the PR checklist, we follow semver, which requires changing the minor version when adding public API.

stuartmorgan-g · 2025-06-12T12:10:00Z

packages/google_maps_flutter/google_maps_flutter_android/example/pubspec.yaml

All of the google_maps_flutter_android and google_maps_flutter_ios changes can be reverted.

stuartmorgan-g · 2025-06-12T12:10:46Z

packages/google_maps_flutter/google_maps_flutter_platform_interface/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 2.11.2
+
+* Adds support to camera control button on web.


"Adds support for disabling or moving the camera control button on web."

stuartmorgan-g · 2025-06-12T12:11:02Z

...gle_maps_flutter/google_maps_flutter_platform_interface/lib/src/types/map_configuration.dart

@@ -44,6 +46,16 @@ class MapConfiguration {
  /// See [WebGestureHandling] for more details.
  final WebGestureHandling? webGestureHandling;

+  /// This setting controls how the API handles cameraControl button position on the map. Web only.
+  ///
+  /// See [WebCameraControlPosition] for more details.


Same comment here; the null behavior should be documented.

stuartmorgan-g · 2025-06-12T12:14:28Z

...lutter/google_maps_flutter_platform_interface/lib/src/types/web_camera_control_position.dart

+| LB          RB | 
+| BL    BC    BR | 
+----------------+
+*/


What's the purpose of duplicating this from the link below? It's not part of the doc comment, and the plugin will never be responsible for actually implementing this behavior, so it's not clear to me who the audience for this comment is.

Good point. I removed the duplicated info.

stuartmorgan-g · 2025-06-12T12:20:40Z

packages/google_maps_flutter/google_maps_flutter_platform_interface/pubspec.yaml

@@ -4,7 +4,7 @@ repository: https://github.com/flutter/packages/tree/main/packages/google_maps_f
 issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+maps%22
 # NOTE: We strongly prefer non-breaking changes, even at the expense of a
 # less-clean API. See https://flutter.dev/go/platform-interface-breaking-changes
-version: 2.11.1
+version: 2.11.2


Same versioning note here.

stuartmorgan-g · 2025-07-29T18:26:52Z

@4rthurmonteiro Are you still planning on updating this based on the feedback above and then splitting out the first sub-PR?

4rthurmonteiro · 2025-07-29T19:41:18Z

@4rthurmonteiro Are you still planning on updating this based on the feedback above and then splitting out the first sub-PR?

Hi @stuartmorgan-g yes!

…into feat/camera_control

…able & position on web (#9897) Platform interface portion of #9089. Part of flutter/flutter#167137 ## Pre-Review Checklist **Note**: The Flutter team is currently trialing the use of [Gemini Code Assist for GitHub](https://developers.google.com/gemini-code-assist/docs/review-github-code). Comments from the `gemini-code-assist` bot should not be taken as authoritative feedback from the Flutter team. If you find its comments useful you can update your code accordingly, but if you are unsure or disagree with the feedback, please feel free to wait for a Flutter team member's review for guidance on which automated comments should be addressed. [^1]: Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling.

4rthurmonteiro · 2025-08-28T19:16:30Z

Hey @stuartmorgan-g and @harryterkelsen the platform web PR is here #9921

… on web (#9921) Platform web portion of #9089. Part of flutter/flutter#167137 ## Pre-Review Checklist **Note**: The Flutter team is currently trialing the use of [Gemini Code Assist for GitHub](https://developers.google.com/gemini-code-assist/docs/review-github-code). Comments from the `gemini-code-assist` bot should not be taken as authoritative feedback from the Flutter team. If you find its comments useful you can update your code accordingly, but if you are unsure or disagree with the feedback, please feel free to wait for a Flutter team member's review for guidance on which automated comments should be addressed. [^1]: Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling.

4rthurmonteiro · 2025-09-02T22:39:19Z

Hey @stuartmorgan-g & @harryterkelsen I updated the PR with new platform-interface version & web version.

stuartmorgan-g

LGTM

…sition on web (flutter/packages#9089)

flutter/packages@5d785a0...42bb347 2025-09-03 [email protected] [google_maps_flutter] Fixes exception when dispose is called while asynchronous update from didUpdateWidget is executed (flutter/packages#9227) 2025-09-03 [email protected] [google_maps_flutter] Add cameraControl enable/disable & position on web (flutter/packages#9089) 2025-09-03 [email protected] [tool] Only license-check checked-in files (flutter/packages#9905) 2025-09-03 [email protected] [interactive_media_ads] Fixes preloading ad while another was playing on Android (flutter/packages#9904) 2025-09-02 [email protected] Update FWFWebViewFlutterWKWebViewExternalAPITests.swift (flutter/packages#9922) 2025-09-02 [email protected] Update Stubs for FlutterPluginRegistrar interface change (flutter/packages#9923) 2025-09-02 [email protected] [camera_android_camerax] Update Android minimum SDK version (flutter/packages#9945) 2025-09-02 [email protected] Roll Flutter from da5523a to 6b18740 (49 revisions) (flutter/packages#9926) 2025-09-02 [email protected] [various] Scrub pre-iOS-13 code (flutter/packages#9849) 2025-09-02 [email protected] [google_maps_flutter_web] Add cameraControl enable/disable & position on web (flutter/packages#9921) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages-flutter-autoroll Please CC [email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

4rthurmonteiro added 3 commits April 15, 2025 15:34

feat: camera control on web

b4cc5f8

chore: update map_configuration_test.dart

9281be8

chore: add dependecy_overrides

25020bf

github-actions bot added p: google_maps_flutter platform-web labels Apr 15, 2025

4rthurmonteiro added 2 commits April 21, 2025 15:52

Merge remote-tracking branch 'upstream/main' into feat/camera_control

f2fe01d

chore(step-1): make-deps-path-based

b2d1d89

github-actions bot added platform-android platform-ios labels Apr 21, 2025

chore(changelog): merge with upstream

a5a5ff5

4rthurmonteiro marked this pull request as ready for review April 21, 2025 19:16

4rthurmonteiro requested review from cbracken, reidbaker, ditman and stuartmorgan-g as code owners April 21, 2025 19:16

Merge branch 'main' into feat/camera_control

14a12bf

4rthurmonteiro requested a review from vashworth as a code owner April 24, 2025 00:49

stuartmorgan-g added the triage-web Should be looked at in web triage label May 13, 2025

mdebbar requested a review from harryterkelsen June 11, 2025 18:09

harryterkelsen reviewed Jun 11, 2025

View reviewed changes

harryterkelsen approved these changes Jun 11, 2025

View reviewed changes

stuartmorgan-g requested changes Jun 12, 2025

View reviewed changes

stuartmorgan-g removed the triage-web Should be looked at in web triage label Jun 12, 2025

4rthurmonteiro added 5 commits August 6, 2025 19:43

Merge branch 'main' into feat/camera_control

59403c3

chore: fix minor

a674307

chore: improve webCameraControlEnabled description

67f8332

chore: add doc for WebCameraControlPosition? when is null

89fc4c4

chore: update changelog

34bdf2c

4rthurmonteiro added 4 commits August 27, 2025 16:28

chore: update docs

a53dea0

Merge branch 'google_maps_flutter_platform_interface/camera_control' …

bfa3f44

…into feat/camera_control

Merge branch 'main' into feat/camera_control

2296090

chore: add webCameraControlPosition test

aff20e2

4rthurmonteiro added 3 commits August 28, 2025 15:02

Merge remote-tracking branch 'upstream/main' into feat/camera_control

e3a17f9

chore: remove platform interface override

a4c3b7c

chore: remove platform interface override

bd44dad

4rthurmonteiro mentioned this pull request Aug 28, 2025

[google_maps_flutter_web] Add cameraControl enable/disable & position on web #9921

Merged

11 tasks

vashworth removed their request for review August 28, 2025 21:44

4rthurmonteiro added 5 commits September 2, 2025 19:24

Merge remote-tracking branch 'upstream/main' into feat/camera_control

347538e

chore: remove dependency override

e11955e

chore: undo web example

ba9375e

chore: undo line change

6c54f61

chore: restore file

c7bf6d7

github-actions bot removed the platform-web label Sep 2, 2025

4rthurmonteiro added 2 commits September 3, 2025 10:25

Merge branch 'main' into feat/camera_control

eba680c

chore: format file

9d7d258

stuartmorgan-g removed request for cbracken, reidbaker and ditman September 3, 2025 13:44

stuartmorgan-g approved these changes Sep 3, 2025

View reviewed changes

stuartmorgan-g added the autosubmit Merge PR when tree becomes green via auto submit App label Sep 3, 2025

auto-submit bot merged commit dc48f5c into flutter:main Sep 3, 2025
80 checks passed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 3, 2025

dc48f5cf7 [google_maps_flutter] Add cameraControl enable/disable & po…

b250e0f

…sition on web (flutter/packages#9089)

engine-flutter-autoroll mentioned this pull request Sep 3, 2025

Roll Packages from 5d785a06c091 to 42bb34706103 (10 revisions) flutter/flutter#174876

Merged

		@@ -1,3 +1,7 @@
		## 2.11.2

		* Adds support to camera control button on web.

[google_maps_flutter] Add cameraControl enable/disable & position on web #9089

[google_maps_flutter] Add cameraControl enable/disable & position on web #9089

Conversation

4rthurmonteiro commented Apr 15, 2025 • edited by stuartmorgan-g Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Pre-Review Checklist

Footnotes

Uh oh!

harryterkelsen left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

harryterkelsen left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

stuartmorgan-g commented Jul 29, 2025

Uh oh!

4rthurmonteiro commented Jul 29, 2025

Uh oh!

4rthurmonteiro commented Aug 28, 2025

Uh oh!

4rthurmonteiro commented Sep 2, 2025

Uh oh!

stuartmorgan-g left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

4rthurmonteiro commented Apr 15, 2025 •

edited by stuartmorgan-g

Loading